Skip to content

Fix TLS access in JIT with Musl #13927

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Apr 11, 2024
Merged

Fix TLS access in JIT with Musl #13927

merged 1 commit into from
Apr 11, 2024
+1 −1

Conversation

arnaud-lb
Copy link
Member

@arnaud-lb arnaud-lb commented Apr 9, 2024 •
edited
Loading

ABI specifies that each DTV element is 8 bytes in size on x86-64. The factor 16 was defined in 71cccc0, but it's probably a merge error of 94ba883, which used the right value.

@arnaud-lb
Fix TLS access in JIT with Musl
b040ea8 
ABI specifies that each DTV element is 8 bytes in size on x86-64. The factor 16
was defined in from 71cccc0, but it's probably
a merge error of 94ba883, which used the right
value.
This was referenced Apr 9, 2024
Closed
Merged
@arnaud-lb arnaud-lb marked this pull request as ready for review April 10, 2024 11:10
@arnaud-lb arnaud-lb merged commit 91b8457 into php:master Apr 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dstogov dstogov dstogov approved these changes

@iluuu1994 iluuu1994 Awaiting requested review from iluuu1994

Assignees
No one assigned
Labels
Extension: opcache OS: Alpine / Musl
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@arnaud-lb @dstogov